Skip to content
This repository has been archived by the owner on Mar 27, 2023. It is now read-only.

fix(planning_launch): lane change collision check default #652

Closed
wants to merge 2 commits into from
Closed

fix(planning_launch): lane change collision check default #652

wants to merge 2 commits into from

Conversation

zulfaqar-azmi-t4
Copy link
Contributor

PR Type

  • Bug Fix

Related Links

fix(behavior_path_planner): lane change check default parameters #2575

Description

To enhance current CI lane change scenario, the parameters are modified.

Review Procedure

Remarks

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

  • Commits are properly organized and messages are according to the guideline
  • Code follows coding guidelines
  • (Optional) Unit tests have been written for new behavior
  • PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

  • All open points are addressed and tracked via issues or tickets
  • Write release notes

CI Checks

  • Build and test for PR: Required to pass before the merge.
  • Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

zulfaqar-azmi-t4 added a commit that referenced this pull request Dec 23, 2022
Signed-off-by: Muhammad Zulfaqar Azmi <[email protected]>
zulfaqar-azmi-t4 added a commit that referenced this pull request Dec 23, 2022
* feat(planning_launch): new lane change path generation parameters

Signed-off-by: Muhammad Zulfaqar Azmi <[email protected]>

* rearrange parameters

Signed-off-by: Muhammad Zulfaqar Azmi <[email protected]>

* fix parameter for stable result

Signed-off-by: Muhammad Zulfaqar Azmi <[email protected]>

* Preparing for #652

Signed-off-by: Muhammad Zulfaqar Azmi <[email protected]>

Signed-off-by: Muhammad Zulfaqar Azmi <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant